-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ServerEngine #880
Add ServerEngine #880
Conversation
Could you add remaining list on the description? |
I got it. |
remaining is: |
Here is GitHub. |
Now, the main signal handler in fluentd and serverengine is like below.
Then I fixed As a result, the signal handler will be same as ordinal fluentd like below.
|
@repeatedly |
5b2db6c
to
b744a53
Compare
[MEMO] The remaining problem is |
$log = Fluent::Log.new(STDERR, Fluent::Log::LEVEL_TRACE) | ||
dl_opts = {} | ||
dl_opts[:log_level] = Fluent::Log::LEVEL_TRACE | ||
logdev = Fluent::Test::DummyLogDevice.new |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fluent::Test::DummyLogDevice
is just for tests. Use STDERR
as before.
9ffcc6a
to
a5f1307
Compare
5ceeb02
to
68448fe
Compare
This looks ready to merge. |
Done! |
Thank you! |
https://github.com/fluent/fluentd/wiki/Fluentd-running-on-ServerEngine
implement
・Logging
・Signal handler
・Daemonize
・Worker processes